Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

utils: load_yaml_data(): only load yaml files #144

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

utils: load_yaml_data(): only load yaml files #144

wants to merge 1 commit into from

Conversation

Rabenherz112
Copy link
Contributor

In the load_yaml_data function we now filter for files with the .yml or .yaml extension. Files which are not yaml files are ignored if possible, an error is only thrown if the file is directly tried to be loaded.

DEBUG:utils.py: loading data from awesome-selfhosted-data/software/writing.yml
DEBUG:utils.py: skipping file wrong-file-extention.xml, not a YAML file
DEBUG:utils.py: loading data from awesome-selfhosted-data/software/xandikos.yml

ERROR:utils.py: file awesome-selfhosted-data/software/wrong-file-extention.xml is not a YAML file

Closes #139

@Rabenherz112
utils: load_yaml_data(): only load yaml files
5d482b7 
In the `load_yaml_data` function we now filter for files with the `.yml` or `.yaml` extension. Files which are not yaml files are ignored if possible, an error is only throw if the file is directly tried to be loaded.
Closes nodiscc#139
@Rabenherz112 Rabenherz112 mentioned this pull request Jan 8, 2025
Closed
@nodiscc nodiscc self-requested a review January 8, 2025 23:17
@nodiscc nodiscc added this to the 1.3.0 milestone Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nodiscc nodiscc Awaiting requested review from nodiscc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.

awesome_lint/markdown*: ignore files without *.yml extension in software/,tags/, ...
2 participants
@Rabenherz112 @nodiscc